Skip to content

Rust SDK #243

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 19 commits into
base: main
Choose a base branch
from
Draft

Rust SDK #243

wants to merge 19 commits into from
+6,860 −0

Conversation

wdoppenberg
Copy link

Hi, this is done in reference to #239 . I started with converting the Typescript SDK's types and events to Rust, maintaining the same packages / crates structure.

The client library implementation will probably cause some changes to ag-ui-core going forward.

Feedback is most welcome.

@vercel Vercel
Copy link

vercel bot commented Aug 4, 2025

@wdoppenberg is attempting to deploy a commit to the CopilotKit Team on Vercel.

A member of the Team first needs to authorize it.

@wdoppenberg wdoppenberg changed the title Rust SDK implementation Rust SDK Aug 4, 2025
tylerslaton
tylerslaton requested changes Sep 17, 2025
View reviewed changes
Copy link
Contributor

@tylerslaton tylerslaton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @wdoppenberg!

Just a couple of asks here.

  1. Can you please add documentation to our site? You can reference #339 and #316 for reference on how.
  2. Can you please move this in to the sdks/community/rust folder? You can use #339 as a reference for that as well.
  3. Could you have it appear as test / Rust SDK Tests and test / Rust SDK Lint?

With those changes we would be happy to merge this! One final question, for the Rust ecosystem how does packaging work? Is it Github based or do we need to submit it to a registry?

@wdoppenberg
Copy link
Author

Hi @wdoppenberg!

Just a couple of asks here.

1. Can you please add documentation to our site? You can reference #339 and #316 for reference on how.

2. Can you please move this in to the `sdks/community/rust` folder? You can use #339 as a reference for that as well.

3. Could you have it appear as `test / Rust SDK Tests` and `test / Rust SDK Lint`?

With those changes we would be happy to merge this! One final question, for the Rust ecosystem how does packaging work? Is it Github based or do we need to submit it to a registry?

Will do!

As for the registry: Rust uses https://crates.io/ as a package registry. I've taken the liberty to secure the following namespaces: https://crates.io/crates/ag-ui-core and https://crates.io/crates/ag-ui-client. I can possibly also secure ag-ui and have these packages exist as submodules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tylerslaton tylerslaton tylerslaton requested changes

@mme mme Awaiting requested review from mme mme will be requested when the pull request is marked ready for review mme is a code owner

@ranst91 ranst91 Awaiting requested review from ranst91 ranst91 will be requested when the pull request is marked ready for review ranst91 is a code owner

@ataibarkai ataibarkai Awaiting requested review from ataibarkai ataibarkai will be requested when the pull request is marked ready for review ataibarkai is a code owner

@maxkorp maxkorp Awaiting requested review from maxkorp maxkorp will be requested when the pull request is marked ready for review maxkorp is a code owner

@NathanTarbert NathanTarbert Awaiting requested review from NathanTarbert NathanTarbert will be requested when the pull request is marked ready for review NathanTarbert is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wdoppenberg @tylerslaton